Fixed the test

Fixed tests and removed the unnecessary comment

Fixed tests

Added tests for building an example as a library

Fixed style issues

Merge branch 'master' of https://github.com/rust-lang/cargo into allow-examples-to-be-library

Allow examples to be libraries

Auto merge of #3542 - creativcoder:check-digit-name, r=alexcrichton

Add a check for names starting with a digit

According to Rust grammer https://doc.rust-lang.org/grammar.html#extern-crate-declarations for extern crate declarations, a crate name cannot start with a digit.

But, currently this rule is not upheld by `cargo new` as creating a project like:

`cargo new 2048` would create a project named 2048 which obviously won't compile with crate declaration like `extern crate 2048` by a consumer.

This obviously is a rare case in practice, but its always good to check i guess.

This PR adds a check to the `new` method and `bail`s out with a message for any names starting with a digit.

PS: I noticed it while making a 2048 puzzle game as a library so thought it would be nice to add this check :)

Add a check for names starting with a digit

Auto merge of #3534 - sfackler:opt-level, r=alexcrichton

Allow debuginfo level to be specified

`true` is mapped to `2`, which matches current behavior.

r? @alexcrichton

Allow debuginfo level to be specified

`true` is mapped to `2`, which matches current behavior.

Auto merge of #3533 - jtgeibel:3519-dev-target, r=alexcrichton

Rename debug target to dev in build output

This renames the debug target to dev in the build output for non-release
builds.  This includes the output of commands such as `cargo build`,
`cargo test`, and `cargo doc`.

Closes #3519

Auto merge of #3532 - alexcrichton:ignore, r=brson

Ignore spuriously failing Windows test

I really have no clue why this test is failing on Windows, and after months of
being unable to diagnose I'm tired of retrying PRs due to this failure. Let's
just ignore it on Windows.

Closes #3466

Auto merge of #3490 - alexcrichton:dev-dep-doc-test, r=brson

Fix cargo test --doc with dev-deps

Previously Cargo accidentally didn't pull in dev-dependencies due to the way
`cargo test --doc` was interpreted in terms of top-level targets. This PR
special cases this situation by ensuring that the doctest intention makes its
way all to the backend and the dependencies can be correctly calculated.

Closes #3422

Rename debug target to dev in build output

This renames the debug target to dev in the build output for non-release
builds.  This includes the output of commands such as `cargo build`,
`cargo test`, and `cargo doc`.

Closes #3519

Auto merge of #3478 - alexcrichton:lift, r=brson

Lift up workspace rlibs while building

I think the condition here was slightly off from before, so invert it subtly to
get what we want, lifting up anything in a workspace or binaries otherwise.

Closes #3432

Ignore spuriously failing Windows test

I really have no clue why this test is failing on Windows, and after months of
being unable to diagnose I'm tired of retrying PRs due to this failure. Let's
just ignore it on Windows.

Closes #3466

Auto merge of #3527 - nikomatsakis:master, r=alexcrichton

check for `CARGO_INCREMENTAL` and pass `-Zincremental` if present

Per the discussion on IRC, this adds a very simple way for cargo users to opt into incremental compilation by setting the `CARGO_INCREMENTAL` environment variable (i.e., `CARGO_INCREMENTAL=1 cargo build`). This will result in incremental data being stored into the `target/incremental` directory. Since it supplies `-Z`, this option is only intended for use on nightly compilers, though cargo makes no effort to check.

The plan is to keep incremental compilation optional until we are feeling more confident it's not going to cause problems for people. At that point, it should become part of the compilation profile. It will be the default when building in debug builds, and opt-in for release builds.

address suggestions from acrichto

check `cargo test` too`

check for `CARGO_INCREMENTAL` and pass `-Zincremental` if present

Auto merge of #3524 - sdroege:test-build-doc-test-check-status, r=alexcrichton

Check the exit code of all processes started in the build/test tests

By not checking the exit codes, a failure in the build::crate_env_vars()
test was ignored.

As suggested by @alexcrichton in https://github.com/rust-lang/cargo/pull/3515#discussion_r95253461

Check the exit code of all processes started in the build/test tests

By not checking the exit codes, a failure in the build::crate_env_vars()
test was ignored.

Auto merge of #3489 - matklad:rel-ws, r=alexcrichton

Use canonical paths for checking equality

Auto merge of #3515 - sdroege:doc-all, r=alexcrichton

 Add support for documenting all members of the workspace with "doc --all"

https://github.com/rust-lang/cargo/issues/3491

Auto merge of #3518 - Manishearth:static-mut, r=alexcrichton

Replace static mut bool with atomicbool

unsafe in tests is fine, but it can be trivially avoided in this case.

r? @alexcrichton

cc @froydnj

Replace static mut bool with atomicbool

Collect library/binary names when documenting even if a spec was given

This seems more consistent and generally should work better.

Check if the "doc --all" tests exit with exit code 0

Auto merge of #3511 - sdroege:build-all, r=alexcrichton

Add support for building all members of the workspace with "build --all"

https://github.com/rust-lang/cargo/issues/3491

Auto merge of #3510 - froydnj:cross-compilation-test-checking, r=alexcrichton

handle cross-compilation test failure more gracefully

The current cross-compilation test module fails every test if an appropriate cross standard library is not found.  This behavior is unhelpful, because the failures are verbose, unexpected, and there's no
obvious way to make them go away.

Instead, it would be better to check once before all the tests that an appropriate cross-compilation setup is available.  Once that check has been done, a single test can fail with an appropriate `panic!` message while the other tests silently pass.  The `panic!` message can inform the user about their options, either setting an appropriate environment variable, or using rustup to install the necessary cross standard library.

Assuming the user has rustup installed, the single failure now looks
something like:

```
thread 'plugin_deps' panicked at 'Cannot cross compile to i686-unknown-linux-gnu.

This failure can be safely ignored. If you would prefer to not see this
failure, you can set the environment variable CFG_DISABLE_CROSS_TESTS to "1".

Alternatively, you can install the necessary libraries for cross-compilation with

    rustup toolchain install stable-i686-unknown-linux-gnu
', tests/cross-compile.rs:87
note: Run with `RUST_BACKTRACE=1` for a backtrace.
```

The code is admittedly gnarly, with synchronization and `unsafe` sections, but I wasn't sure how to make it any better. Suggestions welcome!

Fixes #3086.

Add tests for "build --all"

These are basically the same as the ones from "test --all"

Add tests for "doc --all"

These are basically the same as the ones from "test --all" and "build --all"

Add support for documenting all members of the workspace with "doc --all"

https://github.com/rust-lang/cargo/issues/3491

Normalize paths before checking for equality

handle cross-compilation test failure more gracefully

The current cross-compilation test module fails every test if an
appropriate cross standard library is not found.  This behavior is
unhelpful, because the failures are verbose, unexpected, and there's no
obvious way to make them go away.

Instead, it would be better to check once before all the tests that an
appropriate cross-compilation setup is available.  Once that check has
been done, a single test can fail with an appropriate `panic!` message
while the other tests silently pass.  The `panic!` message can inform
the user about their options, either setting an appropriate environment
variable, or using rustup to install the necessary cross standard
library.

Assuming the user has rustup installed, the single failure now looks
something like:

```
thread 'plugin_deps' panicked at 'Cannot cross compile to i686-unknown-linux-gnu.

This failure can be safely ignored. If you would prefer to not see this
failure, you can set the environment variable CFG_DISABLE_CROSS_TESTS to "1".

Alternatively, you can install the necessary libraries for cross-compilation with

    rustup toolchain install stable-i686-unknown-linux-gnu

You may need to install runtime libraries for your Linux distribution as well.
', tests/cross-compile.rs:87
note: Run with `RUST_BACKTRACE=1` for a backtrace.
```

Fixes #3086.

Auto merge of #3509 - froydnj:colored-shell-tests, r=alexcrichton

disable color shell tests for terminals that don't support color

Some terminals (e.g. running a shell inside Emacs's shell-mode) don't
support color, and running tests that assume the terminal supports color
don't work so well.  Instead, if color is expected, check the terminal
for whether it supports color or not, and act accordingly.

Add support for building all members of the workspace with "build --all"

https://github.com/rust-lang/cargo/issues/3491

Auto merge of #3508 - petrochenkov:bsover, r=alexcrichton

Do not treat `warning` and `rerun-if-changed` as user-defined metadata in build script overrides

Fixes https://github.com/rust-lang/cargo/pull/3438#discussion_r94018355
r? @alexcrichton

disable color shell tests for terminals that don't support color

Some terminals (e.g. running a shell inside Emacs's shell-mode) don't
support color, and running tests that assume the terminal supports color
don't work so well.  Instead, if color is expected, check the terminal
for whether it supports color or not, and act accordingly.

Auto merge of #3504 - froydnj:win32-dependencies, r=alexcrichton

make certain dependencies Windows-only

Inspired by servo/heapsize#71, we make certain dependencies only required under Windows.

Do not treat `warning` and `rerun-if-changed` as user-defined metadata in build script overrides

make certain dependencies Windows-only

Inspired by servo/heapsize#71, we make certain dependencies only
required under Windows.

Auto merge of #3427 - nrc:api-check-2, r=alexcrichton

Make some aspects of check/build available as an API.

There are two key parts to this commit:
* let API clients run `cargo check` with minimal fuss (ops/cargo_check.rs),
* let API clients intercept and customise Cargo's calls to rustc (all the Executor stuff).

r? @alexcrichton

Auto merge of #3505 - froydnj:read2-delete-try, r=alexcrichton

delete unused try macro from read2.rs

AFAICS, this is unused.

Auto merge of #3507 - alexcrichton:appveyor-new-deploy, r=alexcrichton

Remove extraneous .BaseName from AppVeyor

Apparently this isn't necessary

Remove extraneous .BaseName from AppVeyor

Apparently this isn't necessary

Add `--profile check` to `cargo rustc`

delete unused try macro from read2.rs

Auto merge of #3495 - sfackler:bump-shell-escape, r=alexcrichton

Update shell-escape

r? @alexcrichton

Auto merge of #3500 - alexcrichton:appveyor-new-deploy, r=alexcrichton

Try using before_deploy on AppVeyor

Try using before_deploy on AppVeyor

Fix cargo test --doc with dev-deps

Previously Cargo accidentally didn't pull in dev-dependencies due to the way
`cargo test --doc` was interpreted in terms of top-level targets. This PR
special cases this situation by ensuring that the doctest intention makes its
way all to the backend and the dependencies can be correctly calculated.

Closes #3422

Lift up workspace rlibs while building

I think the condition here was slightly off from before, so invert it subtly to
get what we want, lifting up anything in a workspace or binaries otherwise.

Closes #3432

Update shell-escape

review changes, round 2 part b

Remove the ContinueBuild concept, be lenient about missing files in all cases, and undo the fingerprint changes (not an issue now we're not stopping the build early).

review changes, round 2

Mostly focussing on the ergonomics of the API - removes with_* methods with closures, and replaces generics with trait objects.

Review changes

Make some aspects of check/build available as an API.

There are two key parts to this commit:
* let API clients run `cargo check` with minimal fuss (ops/cargo_check.rs),
* let API clients intercept and customise Cargo's calls to rustc (all the Executor stuff).

Auto merge of #3468 - nrc:metadata-emit, r=alexcrichton

cargo check: use --emit=metadata rather than --crate-type=metadata

Requires https://github.com/rust-lang/rust/pull/38571 (don't land before that does)

r? @alexcrichton

Fixup tests

Use --emit=metadata rather than --crate-type=metadata

Requires https://github.com/rust-lang/rust/pull/38571

Auto merge of #3485 - sfackler:useragent, r=alexcrichton

Set a user agent with the version when talking to the registry

This can be used to track what versions of Cargo are being used in the
wild, which can be very useful.

r? @alexcrichton

Set a user agent with the version when talking to the registry

This can be used to track what versions of Cargo are being used in the
wild, which can be very useful.

Auto merge of #3480 - alexcrichton:one-flaky-test, r=brson

Protect against spurious failure in ctrl_c test

A failure was witnessed in the Rust repository [1] which happened right after
this test and was a problem removing a directory. Local testing confirms that if
you kill Cargo then right afterwards it's very unlikely to be able to remove the
build directory, presumably because the child process is still getting torn down
in the background.

This commit fixes the ctrl_c test itself to wait for itself to release the bulid
directory, at which point the test has definitely passed.

[1]: https://ci.appveyor.com/project/rust-lang/rust/build/1.0.1331/job/xq4ogmglj7sllibw

Protect against spurious failure in ctrl_c test

A failure was witnessed in the Rust repository [1] which happened right after
this test and was a problem removing a directory. Local testing confirms that if
you kill Cargo then right afterwards it's very unlikely to be able to remove the
build directory, presumably because the child process is still getting torn down
in the background.

This commit fixes the ctrl_c test itself to wait for itself to release the bulid
directory, at which point the test has definitely passed.

[1]: https://ci.appveyor.com/project/rust-lang/rust/build/1.0.1331/job/xq4ogmglj7sllibw

Auto merge of #3477 - sfackler:shell-escape, r=alexcrichton

Use externally sourced shell-escape

Closes #3374

r? @alexcrichton

Use externally sourced shell-escape

Closes #3374

Auto merge of #3470 - alexcrichton:no-more-urls, r=steveklabnik

Delete manual installation URLs

These have existed since the beginning of time, but they haven't needed to exist
since long before 1.0. Let's just delete them now that they're out of date
anyway.

Delete manual installation URLs

These have existed since the beginning of time, but they haven't needed to exist
since long before 1.0. Let's just delete them now that they're out of date
anyway.

Auto merge of #3435 - alexcrichton:bump-libssh2, r=brson

Update libssh2 to fix a segfault on Windows

There's some more discussion on #3401, but this essentially is just an inclusion
of libssh2/libssh2#163

Closes #3401

Auto merge of #3417 - alexcrichton:fix-svn, r=brson

Ignore dot dirs in directory sources

Looks like they'll conflict with a VCS, so let's just skip them like we do in
directory traversal elsewhere.

Closes #3414

Auto merge of #3412 - alexcrichton:fix-recursion, r=brson

Fix recursion in build_path_deps

We were slightly too eager to follow pointers!

Closes #3330

Auto merge of #3467 - alexcrichton:verbose, r=alexcrichton

Attempt to debug a problem on AppVeyor

Let's get verbose logs from this step

Attempt to debug a problem on AppVeyor

Let's get verbose logs from this step

Auto merge of #3438 - petrochenkov:bsdoc, r=alexcrichton

Add some more details to build script docs

r? @alexcrichton

Add some more details to build script docs

Auto merge of #3458 - alexcrichton:docs, r=steveklabnik

Update README build instructions

Closes #3437

Auto merge of #3454 - alexcrichton:bump1, r=alexcrichton

Bump crates-io to 0.6.0

Auto merge of #3439 - gentoo90:bashcomp, r=alexcrichton

Bashcomp

Update bash completion

Auto merge of #3426 - nrc:check-fix-2, r=alexcrichton

Test for #3419

Based on top of https://github.com/rust-lang/cargo/pull/3425.

I'm not sure if this is an acceptable test because it downloads rustc-serialize.

r? @alexcrichton

Auto merge of #3436 - matklad:abs-target, r=alexcrichton

Make sure targets always use absolute path

Closes #3416

I'd want to add a test for it later. Seems a bit suspicious that this change has not caused any test to fail.

Update README build instructions

Auto merge of #3457 - alexcrichton:no-check-on-beta, r=alexcrichton

Only run check tests on nightly

Only run check tests on nightly

Bump crates-io to 0.6.0

Test for cargo check with macro and both lib and bin.

Closes #3419

Add a test

Includes adding facility for checking that output does not contain a string.

Update bash completion

Unify indentation in bash completion

Make sure targets always use absolute path

Update libssh2 to fix a segfault on Windows

There's some more discussion on #3401, but this essentially is just an inclusion
of libssh2/libssh2#163

Closes #3401

Ignore dot dirs in directory sources

Looks like they'll conflict with a VCS, so let's just skip them like we do in
directory traversal elsewhere.

Closes #3414

Auto merge of #3430 - alexcrichton:bump, r=brson

Bump to 0.17.0

Bump to 0.17.0

Auto merge of #3424 - matklad:naming-things, r=alexcrichton

Minor, more explicit names

Ultraminor renaming. Have to look at this tests because it fails if you link with older libgit.

Auto merge of #3413 - matklad:ref, r=alexcrichton

Refactor resolve_dependencies

This moves `resolve_dependencies` function from `compile.rs` into the `resolve.rs`, where it more reasonably belongs.

Auto merge of #3410 - fmdkdd:fix-3390-message-format-parse-stderr, r=alexcrichton

Fix `--message-format JSON` when rustc emits non-JSON warnings

The `--message-format JSON` flag parses all the stderr output of rustc to JSON,
but rustc can emit non-JSON lines to stderr (e.g., for warning about the
unstable `-Z` flag on the stable channel), causing cargo to fail reporting
compilation errors when using `--message-format JSON`.

This commit adds a check to look for lines beginning with `{` to only parse
these lines as JSON.  Other lines from rustc are forwarded to the stderr of
cargo.

Fixes #3390.

make sure we use the check profile where possible for lib deps.

Fixes #3418

Minor, more explicit names